Skip to content

lighter dependencies in devtools#598

Merged
ms-lolo merged 4 commits intomainfrom
feature/lighter-dependencies
Dec 1, 2025
Merged

lighter dependencies in devtools#598
ms-lolo merged 4 commits intomainfrom
feature/lighter-dependencies

Conversation

@ms-lolo
Copy link
Copy Markdown
Contributor

@ms-lolo ms-lolo commented Dec 1, 2025

removed the twine dependency because it's not longer used, and moved all the documentation related dependencies to the dev group because we want the end user to select the set of packages on their own based on what they decide to put in their mkdocs.yaml config. this does mean that the rats-docs commands will fail if the user hasn't added the mkdocs package as a dependency.

these dependencies are specific to our mkdocs config but we want to allow
the end user to decide on their own set of plugins for this.
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.59%. Comparing base (c78a616) to head (9a80c09).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
Components Coverage Δ
rats-apps ∅ <ø> (∅)
rats-devtools ∅ <ø> (∅)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR reduces the dependency footprint of the rats-devtools package by removing unused dependencies and moving documentation-related dependencies to the dev group. The changes allow end users to choose their own documentation tooling based on their mkdocs.yaml configuration.

Key Changes:

  • Removed twine dependency (no longer used)
  • Moved documentation dependencies (mkdocs, mkdocs-material, etc.) from main dependencies to dev group
  • Version bumped from 0.14.2 to 0.15.0 across all packages

Reviewed changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
rats-devtools/pyproject.toml Removed twine, moved 8 doc-related dependencies to dev group
rats-devtools/uv.lock Updated lock file reflecting dependency changes and removed transitive dependencies
rats/pyproject.toml Version bump to 0.15.0
rats/uv.lock Updated lock file with removed dependencies
rats-apps/pyproject.toml Version bump to 0.15.0
rats-apps/uv.lock Version bump to 0.15.0
.github/workflows/codeql.yaml Removed javascript-typescript from CodeQL analysis (aligns with dependency cleanup)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@ms-lolo ms-lolo merged commit e9b084e into main Dec 1, 2025
19 checks passed
@ms-lolo ms-lolo deleted the feature/lighter-dependencies branch December 1, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants